Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release binaries for v0.8.27 #155

Merged
merged 2 commits into from
Sep 4, 2024
Merged

Release binaries for v0.8.27 #155

merged 2 commits into from
Sep 4, 2024

Conversation

clonker
Copy link
Member

@clonker clonker commented Sep 4, 2024

No description provided.

@clonker clonker merged commit 87538b7 into gh-pages Sep 4, 2024
7 checks passed
@clonker clonker deleted the release-0.8.27 branch September 4, 2024 11:42
Comment on lines +36 to +40
sudo rm -rf /Applications/Xcode_14.3.1.app
sudo rm -rf /Applications/Xcode_15.0.1.app
sudo rm -rf /Applications/Xcode_15.1.app
sudo rm -rf /Applications/Xcode_15.2.app
sudo rm -rf /Applications/Xcode_15.3.app
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this always installed? I'm wondering if this will not break eventually. I.e. when the base macos-latest image stop to provide them (currently it is macos-14): https://github.com/actions/runner-images/blob/main/images/macos/macos-14-Readme.md

Copy link
Member

@r0qs r0qs Sep 4, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we need to create an issue to rethink the cleanup process before the next release.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, you used --force, so it will ignore nonexistent files. So yeah, it will not break.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IMO this shouldn't be more than a temporary fix in any case.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So yeah, it will not break.

Well, it will break, because when it stops deleting stuff, we'll run out of space again :)

Not sure if we can do better though. The macOS runner does not have the extra data partition so to fit the repo in we do have to delete stuff. We should just make this a bit more robust by not depending on specific versions of stuff we delete.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, what I mean by breaking was related to the rm command failing to find the files. But yeah, it will run out of space again for sure. Maybe we could have a script that check for the current disk space and erase old dated files. Not the best approach either, but maybe a bit better than what we currently do.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

An alternative would be to go back to deleting old nightlies. We have so many now that github crashes when you go to bin/. It's just that it also does not feel like a proper solution because we can't delete them from history so we're still storing them there. The repo will still keep growing in an unbounded way, only the checkout will be smaller.

I don't think it was ever a good idea to store nightly binaries in a github repo (they really should have been stored as something less permanent, e.g. pre-releases, if at all).

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's still better than not deleting it b/c you can do a shallow clone (theoretically at least if we ignore the bytecode compare). Is anyone relying on the nightlies being here? Otherwise we could just not store them here anymore and people can fetch them from CI as artifacts?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can access them e.g. in Remix, so some people may be using them. Some frameworks may be making them available to users too - I remember e.g. that there was some interest from Truffle in having them accessible in the new directory structure. But I don't think anyone is relying on them in production. And they do, we don't have to support that (I wonder if e.g. Sourcify does?).

I added this for discussion next Wednesday. Perhaps it's time to drop some historical baggage and simplify things. I'd rather avoid half-measures, but I'm also not sure how far we can safely go.

TBH I wouldn't be against just scrapping those nightlies completely. They're a nuisance while at the same time they are not useful enough. It's often much more convenient to get a platform-specific binary from develop and we don't have those here. And also it's often more useful for us to share a build from an unmerged PR. Both of these things we currently get from CircleCI and we don't even notice when the nightlies here break. CircleCI only stores artifacts for 30 days, but that's usually more than enough. The only annoying bit is that giving them to people always requires explanations, because links are a buried deep in the CCI UI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants